Skip to content

feat: add {random} / {random:N} project name placeholder's - #2431

Merged
richiemcilroy merged 5 commits into
CapSoftware:mainfrom
mynameistito:fix/random-project-name-template
Oct 21, 2026
Merged

richiemcilroy merged 5 commits into
CapSoftware:mainfrom
mynameistito:fix/random-project-name-template

Conversation

@mynameistito

@mynameistito mynameistito commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Closes #2430.

Adds {random} (10 lowercase hexadecimal characters) and {random:N} (1–32 characters) to project-name templates in both desktop recording paths. Documents the placeholders in settings and the GPUI preview behavior.

Validation: Biome and Rust formatting pass. Focused Rust tests could not run in this environment: the GPUI crate depends on a missing local zed-cap checkout, and the Tauri crate's pinned cidre source is unavailable offline.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge, with both previous findings fixed and no new actionable issues found.

Summary

Adds {random} and {random:N} to project-name templates in both desktop recording paths.

  • Recording names can include a random hexadecimal piece.

Reviews (3) · Last reviewed commit: "fix: import chrono timezone in project n..." · Reviewed by Greptile

Comment thread apps/desktop/src-tauri/src/recording.rs Outdated
@mynameistito mynameistito changed the title feat: add random project name placeholder feat: add {random} / {random:N} project name placeholder's Oct 7, 2026
@richiemcilroy

Copy link
Copy Markdown
Member

Thanks @mynameistito! Greptile is at 4/5 here. We only merge once Greptile reaches 5/5, so please address its findings in the summary above (or reply to them if you think one is wrong), then comment @greptileai review to re-run it. If the branch is behind main, please merge or rebase on main first. Once it is 5/5 we will run the full verification suite on it.

@mynameistito

Copy link
Copy Markdown
Contributor Author

@greptile

@mynameistito

Copy link
Copy Markdown
Contributor Author

@greptileai review

@mynameistito

Copy link
Copy Markdown
Contributor Author

Thanks @mynameistito! Greptile is at 4/5 here. We only merge once Greptile reaches 5/5, so please address its findings in the summary above (or reply to them if you think one is wrong), then comment @greptileai review to re-run it. If the branch is behind main, please merge or rebase on main first. Once it is 5/5 we will run the full verification suite on it.

Hey will do!

I believe my last fixed fixed up the greptile issue!

Comment thread apps/desktop/src-tauri/src/recording.rs
@vercel

vercel Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@mynameistito is attempting to deploy a commit to the Cap Software Inc Team on Vercel.

A member of the Team first needs to authorize it.

@mynameistito

Copy link
Copy Markdown
Contributor Author

@greptileai review this

@mynameistito

Copy link
Copy Markdown
Contributor Author

All 5/5 now @richiemcilroy

@richiemcilroy
richiemcilroy merged commit 88bce03 into CapSoftware:main Oct 21, 2026
66 of 76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Add random string placeholder to default project name templates

2 participants